Fix StatefulSet name references in k8s documentation#283
Merged
sfroment merged 2 commits intodev/giwook-Han/sequencer-failoverfrom Dec 3, 2025
Merged
Fix StatefulSet name references in k8s documentation#283sfroment merged 2 commits intodev/giwook-Han/sequencer-failoverfrom
sfroment merged 2 commits intodev/giwook-Han/sequencer-failoverfrom
Conversation
8 tasks
Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Address feedback from PR #271 on leader election implementation
Fix StatefulSet name references in k8s documentation
Dec 3, 2025
sfroment
approved these changes
Dec 3, 2025
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes documentation inconsistencies where the StatefulSet name was incorrectly referenced as mojave-sequencer-deployment when the actual resource name is mojave-sequencer as defined in k8s/stateful.sequencer.yaml.
- Updated kubectl command to use correct StatefulSet name
- Corrected StatefulSet description text
- Fixed PVC naming pattern to reflect proper Kubernetes conventions
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
sfroment
added a commit
that referenced
this pull request
Dec 3, 2025
* init: add k8s yaml * feat: add k8s leader election logic * add: k8s yamls * mod: add k8s leader shutdown * add: k8s node yaml * fix: use pv&pvc to maintain status on k8s * chore: yaml && add docs * feat: mod batch producer load batch number when it starts (#272) * fix: cargo.toml * chore: toml * mod: k8s + ovh * chore: lint Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: fix here we have 2 result imbricated Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: change a bit Signed-off-by: Sacha Froment <sfroment42@gmail.com> * feat: add node deploy Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: fix potential race Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: update docs Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: use anyhow error Signed-off-by: Sacha Froment <sfroment42@gmail.com> * refactor: change the way the main is started" (#275) * chore: add health check Signed-off-by: Sacha Froment <sfroment42@gmail.com> * refactor: change the way the main is started Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: nitpick * mod: k8s yaml * chore: remove command in justfile * chore: fix just sequencer & node , fix kill process in start.sh * chore: rm * refactor: coordination k8s (#276) * refactor: coordination k8s Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: save Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: keep health alive Signed-off-by: Sacha Froment <sfroment42@gmail.com> * fix: start Signed-off-by: Sacha Froment <sfroment42@gmail.com> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * chore: remove useless Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: review comment Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: add back save on shutdown Signed-off-by: Sacha Froment <sfroment42@gmail.com> --------- Signed-off-by: Sacha Froment <sfroment42@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update justfile Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update cmd/sequencer/src/main.rs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update cmd/sequencer/src/cli.rs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update cmd/node/src/cli.rs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update cmd/sequencer/src/cli.rs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update justfile Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * chore: fix 1st call Signed-off-by: Sacha Froment <sfroment42@gmail.com> * Refactor: Use idiomatic assert! in CLI tests (#277) * Initial plan * Replace if-panic patterns with assert! in CLI tests Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> * Fix missing assert in CLI test for command parsing (#278) * Initial plan * Fix matches! macro usage in sequencer CLI tests Added assert! around matches! calls in parse_stop_and_get_pub_key test to properly verify command types instead of just evaluating the boolean. Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Signed-off-by: Sacha Froment <sfroment42@gmail.com> Co-authored-by: Giwook-Han <hkw00011@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> * chore: lint Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: lint Signed-off-by: Sacha Froment <sfroment42@gmail.com> * chore: fix registry Signed-off-by: Sacha Froment <sfroment42@gmail.com> * Update docs/k8s.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update docs/k8s.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix StatefulSet name references in k8s documentation (#283) * Initial plan * Fix StatefulSet name references in k8s documentation Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> * Fix documentation references to use correct StatefulSet manifest filename (#282) * Initial plan * Fix documentation: update references from deploy.sequencer.yaml to stateful.sequencer.yaml Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> * Fix incorrect filename references in Kubernetes documentation (#281) * Initial plan * Fix all references from deploy.sequencer.yaml to stateful.sequencer.yaml Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> * Consolidate shutdown logic in K8s leader election error handling (#284) * Initial plan * Call shutdown() on error paths to ensure consistent cleanup Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com> * fix: healt.port Signed-off-by: Sacha Froment <sfroment42@gmail.com> --------- Signed-off-by: Sacha Froment <sfroment42@gmail.com> Co-authored-by: Sacha Froment <sfroment42@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: sfroment <7238385+sfroment@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why is this change necessary?
Documentation referenced
mojave-sequencer-deploymentbut the actual StatefulSet name ismojave-sequencerperk8s/stateful.sequencer.yaml.Description
Corrected three mismatched references:
kubectl get sts mojave-sequencer -n 1sixtechsequencer-datadir-mojave-sequencer-0Closes #271
Type of change
Checklist
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.